Thanks for the PR, this is a good step in normalizing certificate fields across the project.

Normalize serial_number output to string format (hex), which is the common way serial number are shown (e.g by browser or openssl).

Unfortunately serial numbers can also be negative (signed) integers (4.1.2.2, https://www.ietf.org/rfc/rfc2459.txt, -set_serial n in OpenSSL), which is difficult to properly communicate in hex notation (-1337 -> FA C7).

We prefer the notation of serial numbers as integers, as most cryptography libraries do by default as well (BoringSSL, asn1crypto, pyca/cryptography).

Perhaps we can add two fields to accommodate both use cases, serial_number and serial_hex or serial_number_hex.

@william-billaud I agree with @JSCU-CNI here. Even though it's only a data representation problem of the same data, I can see the value of having a readily available/searchable hex variant too.

Maybe opt for serial_number_hex?

which is difficult to properly communicate in hex notation (-1337 -> FA C7).

Even more so because Python by default will make that -0x539.

i have added a test with a negative serial number + add serial_number and serial_number_hex field for both plugins.

Codecov Report

❌ Patch coverage is 75.55556% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.74%. Comparing base (a102b31) to head (1539031).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1498      +/-   ##
==========================================
- Coverage   80.74%   80.74%   -0.01%     
==========================================
  Files         394      394              
  Lines       34571    34614      +43     
==========================================
+ Hits        27916    27948      +32     
- Misses       6655     6666      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

CodSpeed Performance Report

Merging this PR will not alter performance

_{Comparing william-billaud:normalize_certlog_output (1539031) with main (a102b31)}

Summary

✅ 11 untouched benchmarks